Remove Sqlite from list of pre-installed providers#48950
Remove Sqlite from list of pre-installed providers#48950kaxil wants to merge 1 commit intoapache:mainfrom
Conversation
Sqlite provider just has 1 hook and isn't needed for `LocalExecutor` to work with Sqlite.
|
Actually I think it is not the best way - I looked more closely - and SO the right way of fixing it is to add another core-dependency |
|
I can open another request for that @kaxil |
|
Actually - not even that. I believe Airflow now needs aiosqlite whenever SQLite connection is used @dstandish - because it will always attempt to create the sqlite async engine. So we really need to make aiosqlite a required dependency for airflow IMHO |
|
Alternative in #48966 -> we need to keep aiosqlite in core. |
|
Closing this one. |
2 participants
Sqlite provider just has 1 hook and isn't needed for
LocalExecutorto work with Sqlite.^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named
{pr_number}.significant.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.